improve and enhance test coverage #242
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Configure Surefire to run API tests (*IT.java) by default - Add 25+ new integration tests (GcpRegion, GlobalFields, Taxonomy) - Update send-report.sh for automated Slack reporting - Add maven-surefire-report-plugin for HTML reports API tests: mvn clean test Unit tests: mvn clean test -Dtest='Test*' jacoco:report
|
We regret to inform you that you are currently not able to merge your changes into the master branch due to restrictions applied by our SRE team. To proceed with merging your changes, we kindly request that you create a pull request from the staging branch. Our team will then review the changes and work with you to ensure a successful merge into the master branch. |
|
We regret to inform you that you are currently not able to merge your changes into the master branch due to restrictions applied by our SRE team. To proceed with merging your changes, we kindly request that you create a pull request from the staging branch. Our team will then review the changes and work with you to ensure a successful merge into the master branch. |
…arious parameters and scenarios
… additional edge cases and validation scenarios.
…, configurations, factory methods, headers, and various edge cases.
…TestStack class, covering various scenarios including disabled live preview, null parameters, and parameter assignment.
harshithad0703
changed the title
…s, JSON handling methods, edge cases, and token validation scenarios.
…uilding methods, method chaining, and various edge cases.
…Hub Actions for unit testing with coverage checks
📊 Unit Test Coverage Report
|
…eneration and verification steps, ensuring coverage reports are created and validating their existence before badge generation.
…ring creation with character array for long values and update list creation to use Arrays.asList for better readability and compatibility.
… paths and adjust test execution parameters to ensure accurate coverage reporting and validation.
…treamline coverage checks
…ge checks with output variables, and add PR comment for coverage summary.
…mprove JaCoCo coverage checks with XML report validation, and ensure consistent execution of coverage summary and report upload steps.
…setup to version 9 with Zulu distribution, streamline coverage extraction from JaCoCo HTML report, and enhance PR comment formatting for coverage summary.
…rin distribution and enable Maven caching.
|
Coverage Summary
|
harshithad0703
changed the title
sunil-lakshman
approved these changes
Nov 10, 2025
reeshika-h
approved these changes
Nov 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
API tests: mvn clean test
Unit tests: mvn clean test -Dtest='Test*' jacoco:report